Skip to content

Correct case for 'pi' in area calculation challenge#54471

Merged
BillWagner merged 2 commits into
mainfrom
mwiemer-microsoft-patch-7
Jun 18, 2026
Merged

Correct case for 'pi' in area calculation challenge#54471
BillWagner merged 2 commits into
mainfrom
mwiemer-microsoft-patch-7

Conversation

@mwiemer-microsoft

@mwiemer-microsoft mwiemer-microsoft commented Jun 17, 2026

Copy link
Copy Markdown
Member

Copilot AI review requested due to automatic review settings June 17, 2026 23:47
@mwiemer-microsoft mwiemer-microsoft requested review from a team and BillWagner as code owners June 17, 2026 23:47
@dotnetrepoman dotnetrepoman Bot added this to the June 2026 milestone Jun 17, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the “numbers in C#” tutorial to use the correct casing for the mathematical constant pi in the circle-area challenge, while still pointing learners to the System.Math.PI constant.

Changes:

  • Replaces “PI” with “pi” in the area-of-a-circle challenge text.
  • Updates the accompanying hint text to refer to “pi” consistently.

Comment thread docs/csharp/tour-of-csharp/tutorials/numbers-in-csharp.md Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

@BillWagner BillWagner left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @mwiemer-microsoft

This LGTM, and I'll :shipit: as soon as the build finishes.

@BillWagner BillWagner enabled auto-merge (squash) June 18, 2026 13:04

@wadepickett wadepickett left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@BillWagner, I reviewed and approved.

@BillWagner BillWagner merged commit 9076a5a into main Jun 18, 2026
11 checks passed
@BillWagner BillWagner deleted the mwiemer-microsoft-patch-7 branch June 18, 2026 18:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants